feat: Update version to 0.6.0 and enhance documentation with release …#42
Conversation
There was a problem hiding this comment.
Pull request overview
Bumps the Java SDK to v0.6.0, updates core dependency/plugin versions, and adds v0.6.0 release notes documenting the transport resilience improvements.
Changes:
- Update SDK version references to
0.6.0(POM, runtimeVersion, README). - Upgrade Arrow Flight/ADBC, Gson, Netty, and multiple Maven build plugins.
- Add
docs/release_notes/v0.6.0.mddescribing the 0.6.0 changes.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
src/main/java/ai/spice/Version.java |
Updates the SDK’s internal version constant to 0.6.0. |
pom.xml |
Bumps project version and upgrades dependencies/plugins (notably Netty). |
docs/release_notes/v0.6.0.md |
Adds v0.6.0 release notes and dependency/plugin tables. |
README.md |
Updates Maven/Gradle install snippets to 0.6.0. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | Plugin | Previous | Current | | ||
| | ------------------------------- | -------- | ------- | | ||
| | maven-surefire-plugin | 3.5.4 | 3.5.5 | | ||
| | maven-source-plugin | 3.3.1 | 3.4.0 | |
There was a problem hiding this comment.
The build plugin table also uses || at the start of each row, which renders as an extra empty column in Markdown. Replace the double pipes with a single | at row starts to produce a standard 3-column table.
| <dependency> | ||
| <groupId>io.netty</groupId> | ||
| <artifactId>netty-all</artifactId> | ||
| <version>4.1.130.Final</version> | ||
| <version>4.2.12.Final</version> | ||
| </dependency> |
There was a problem hiding this comment.
The direct dependency on io.netty:netty-all at 4.2.x is likely to override the Netty version that Apache Arrow Flight/gRPC bring in and can cause runtime classpath conflicts (e.g., NoSuchMethodError) or unvalidated behavior; gRPC’s commonly supported Netty line is 4.1.x. Consider either (1) removing the explicit netty-all dependency and letting Arrow/gRPC manage Netty, or (2) pinning Netty to the same 4.1.x version used by your gRPC/Arrow stack via dependencyManagement, or (3) switching to shaded gRPC Netty artifacts to avoid Netty version coupling.
| | Dependency | Previous | Current | | ||
| | ----------------------------------- | ------------- | ------------ | | ||
| | Apache Arrow Flight SQL | 18.3.0 | 19.0.0 | | ||
| | Apache Arrow ADBC Driver Flight SQL | 0.21.0 | 0.22.0 | |
There was a problem hiding this comment.
The dependency table uses || at the start of each row, which adds an unintended empty column and renders incorrectly in standard Markdown. Use a single leading | for the header and separator rows (and each data row) to match the formatting used in other release notes files.
No description provided.